736: Changed discord message to have leaderboard ID by Arshadul-Monir · Pull Request #733 · tahminator/codebloom

Arshadul-Monir · 2026-02-04T16:43:52Z

736 Description of changes

Added the leaderboard ID to the url message + replaced the hardcoded prod url with ServerUrlUtils url

Checklist before review

I have done a thorough self-review of the PR
Copilot has reviewed my latest changes, and all comments have been fixed and/or closed.
If I have made database changes, I have made sure I followed all the db repo rules listed in the wiki here. (check if no db changes)
All tests have passed
I have successfully deployed this PR to staging
I have done manual QA in both dev (and staging if possible) and attached screenshots below.

Screenshots

Dev

Staging

github-actions · 2026-02-04T16:44:25Z

Available PR Commands

/ai - Triggers all AI review commands at once
/review - AI review of the PR changes
/describe - AI-powered description of the PR
/improve - AI-powered suggestions
/deploy - Deploy to staging

See: https://github.com/tahminator/codebloom/wiki/CI-Commands

github-actions · 2026-02-04T16:45:35Z

Title

736: Changed discord message to have leaderboard ID

PR Type

Bug fix

Description

Include leaderboardId in Discord URLs
Update completed and weekly message templates
Pass currentLeaderboard.getId() to formatter

Diagram Walkthrough

flowchart LR
  DCM["DiscordClubManager.java"]
  Completed["sendLeaderboardCompletedDiscordMessage"]
  Weekly["sendWeeklyLeaderboardUpdateDiscordMessage"]
  URL["/leaderboard/{leaderboardId}?{tag}=true"]

  DCM -- updates --> Completed
  DCM -- updates --> Weekly
  Completed -- formats link --> URL
  Weekly -- formats link --> URL

File Walkthrough

Relevant files

Bug fix

DiscordClubManager.java `Include leaderboardId in Discord leaderboard URLs` src/main/java/org/patinanetwork/codebloom/common/components/DiscordClubManager.java Add leaderboardId to leaderboard URL path. Update completed message link to /leaderboard/%s. Update weekly update link to /leaderboard/%s. Inject currentLeaderboard.getId() into formatter args.	+4/-2

github-actions · 2026-02-04T16:46:56Z

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review Hardcoded URL The leaderboard link still hardcodes 'https://codebloom.patinanetwork.org'. For environment flexibility, construct the link using 'serverUrlUtils.getUrl()' (e.g., '%s/leaderboard/%s?%s=true') so dev/staging/prod all receive correct links. Applies to both the "completed" and "weekly update" messages. View the full leaderboard for %s users at https://codebloom.patinanetwork.org/leaderboard/%s?%s=true Happy LeetCoding, Possible NPE 'currentLeaderboard.getId()' is used without a null-safety guard. Ensure 'currentLeaderboard' is always present when sending these messages or guard against null to avoid a runtime NPE. currentLeaderboard.getId(), club.getTag().name().toLowerCase(),

src/main/java/org/patinanetwork/codebloom/common/components/DiscordClubManager.java

Copilot

Pull request overview

This PR aims to add leaderboard IDs to Discord message URLs and replace hardcoded production URLs with dynamic URLs from ServerUrlUtils. However, the implementation is incomplete - while the leaderboard ID parameter was added, the hardcoded production URL was not replaced with the dynamic URL as stated in the PR description.

Changes:

Added leaderboard ID as a path parameter to Discord message URLs
Added new format placeholders and corresponding parameters for leaderboard IDs in both completion and weekly update messages

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

src/main/java/org/patinanetwork/codebloom/common/components/DiscordClubManager.java

Arshadul-Monir · 2026-02-04T17:04:53Z

/deploy

github-actions · 2026-02-04T17:05:42Z

The command to deploy to staging for the commit 9842ef5 has been triggered. View action run

github-actions · 2026-02-04T17:16:25Z

Staging deployment succeeded for commit 9842ef5

View run

Arshadul-Monir · 2026-02-04T18:04:24Z

/deploy

github-actions · 2026-02-04T18:05:22Z

The command to deploy to staging for the commit 706567b has been triggered. View action run

sonarqubecloud · 2026-02-04T18:06:47Z

Quality Gate passed for 'codebloom_frontend'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

sonarqubecloud · 2026-02-04T18:09:16Z

Quality Gate passed for 'codebloom_backend'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

github-actions · 2026-02-04T18:15:07Z

Staging deployment succeeded for commit 706567b

View run

tahminator · 2026-02-04T18:58:14Z

/copy

github-actions · 2026-02-04T19:00:45Z

Database copy command completed successfully!

Arshadul-Monir · 2026-02-06T04:22:46Z

/deploy

Arshadul-Monir · 2026-02-07T15:11:56Z

/deploy

Arshadul-Monir · 2026-02-07T15:29:21Z

/deploy

Arshadul-Monir · 2026-02-07T15:47:35Z

/deploy

tahminator · 2026-02-08T00:49:25Z

/copy

github-actions · 2026-02-08T00:51:15Z

Database copy command completed successfully!

Arshadul-Monir self-assigned this Feb 4, 2026

Arshadul-Monir requested a review from Copilot February 4, 2026 16:47

Copilot started reviewing on behalf of Arshadul-Monir February 4, 2026 16:47 View session

github-actions bot reviewed Feb 4, 2026

View reviewed changes

src/main/java/org/patinanetwork/codebloom/common/components/DiscordClubManager.java Show resolved Hide resolved

src/main/java/org/patinanetwork/codebloom/common/components/DiscordClubManager.java Show resolved Hide resolved

Copilot AI reviewed Feb 4, 2026

View reviewed changes

Arshadul-Monir force-pushed the 736 branch 2 times, most recently from 04672c8 to 8bc0ff2 Compare February 6, 2026 04:22

Arshadul-Monir force-pushed the 736 branch from 8bc0ff2 to 69f2dcb Compare February 7, 2026 15:29

Arshadul-Monir force-pushed the 736 branch from 69f2dcb to 0676ef4 Compare February 7, 2026 16:07

Arshadul-Monir added 4 commits February 7, 2026 18:38

736: Changed discord message to have leaderboard ID

b51096c

736: Removed hard-coded url by using ServerUrlUtils

cef0487

736: Updated leaderboardChannelId for Patina Network and MHC++

13e9786

736: Removed leaderboard id from weekly message

1ffd67a

Arshadul-Monir force-pushed the 736 branch from 0676ef4 to 1ffd67a Compare February 7, 2026 23:38

736: Updated github script to updated GWC data

64a3b18

Conversation

Arshadul-Monir commented Feb 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

736

Description of changes

Checklist before review

Screenshots

Dev

Staging

Uh oh!

github-actions bot commented Feb 4, 2026

Available PR Commands

Uh oh!

github-actions bot commented Feb 4, 2026

Title

PR Type

Description

Diagram Walkthrough

File Walkthrough

Uh oh!

github-actions bot commented Feb 4, 2026

PR Reviewer Guide 🔍

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Arshadul-Monir commented Feb 4, 2026

Uh oh!

github-actions bot commented Feb 4, 2026

Uh oh!

github-actions bot commented Feb 4, 2026

Uh oh!

Arshadul-Monir commented Feb 4, 2026

Uh oh!

github-actions bot commented Feb 4, 2026

Uh oh!

sonarqubecloud bot commented Feb 4, 2026

Quality Gate passed for 'codebloom_frontend'

Uh oh!

sonarqubecloud bot commented Feb 4, 2026

Quality Gate passed for 'codebloom_backend'

Uh oh!

github-actions bot commented Feb 4, 2026

Uh oh!

tahminator commented Feb 4, 2026

Uh oh!

github-actions bot commented Feb 4, 2026

Uh oh!

Arshadul-Monir commented Feb 6, 2026

Uh oh!

Arshadul-Monir commented Feb 7, 2026

Uh oh!

Arshadul-Monir commented Feb 7, 2026

Uh oh!

Arshadul-Monir commented Feb 7, 2026

Uh oh!

tahminator commented Feb 8, 2026

Uh oh!

github-actions bot commented Feb 8, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Arshadul-Monir commented Feb 4, 2026 •

edited

Loading